Skip to content

docs(dedup): clarify survivor completeness scope - #708

Merged
wesm merged 1 commit into
mainfrom
docs/finalize-dedup-survivor-docs
Sep 2, 2026
Merged

docs(dedup): clarify survivor completeness scope#708
wesm merged 1 commit into
mainfrom
docs/finalize-dedup-survivor-docs

Conversation

@wesm

@wesm wesm commented Aug 29, 2026

Copy link
Copy Markdown
Member

The dedup docs now state the final survivor rule from #703: attachment count, attachment presence, and payload size are considered only when every eligible copy has the same normalized MIME.

This also updates the survivor-selection diagram source and accessible text so future generated assets preserve that group-wide rule. The matching generated diagram is now published on the docs-generated-assets branch.

@roborev-ci

roborev-ci Bot commented Aug 29, 2026

Copy link
Copy Markdown

roborev: Combined Review (9c47f9f)

Verdict: No Medium, High, or Critical findings.

The change is clean at the required severity threshold.


Reviewers: 2 done | Synthesis: codex, 5s | Total: 2m21s

@wesm wesm self-assigned this Sep 2, 2026
Payload completeness is deterministic only when every eligible copy has the
same normalized MIME. The merged documentation still described a pairwise
check, which did not capture the final survivor-selection rule.

Align the written guidance and diagram source with the implemented behavior so
future asset regeneration cannot restore misleading instructions.

Generated with Codex
Co-authored-by: Codex <codex@openai.com>
@wesm
wesm force-pushed the docs/finalize-dedup-survivor-docs branch from 9c47f9f to fefb43a Compare September 2, 2026 10:44
@roborev-ci

roborev-ci Bot commented Sep 2, 2026

Copy link
Copy Markdown

roborev: Combined Review (fefb43a)

No issues found.


Reviewers: 2 done | Synthesis: codex | Total: 50s

@wesm
wesm merged commit f941cad into main Sep 2, 2026
18 checks passed
@wesm
wesm deleted the docs/finalize-dedup-survivor-docs branch September 2, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant